Skip to content

Conversation

@yashsarkar164
Copy link

This PR improves both the analytics service layer and the frontend UI structure without changing any existing functionality.

Backend (analyticsService)

  • Centralized analytics cache keys into a single constant object
  • Centralized cache TTL values to remove magic numbers
  • Added shared helper functions for:
    • Date range calculation
    • Rounding numeric values
  • Reduced duplicated logic in monthly comparison calculations
  • Improved code clarity and future extensibility

Frontend (UI cleanup)

  • Removed duplicate footer markup
  • Removed duplicate PWA install prompt block
  • Removed duplicate script includes
  • Fixed invalid HTML markup that could break layout rendering

Notes

  • No business logic or API behavior has been changed
  • All analytics outputs remain identical
  • UI behavior remains unchanged, only structural cleanup
  • This refactor makes future analytics features easier to add and test

Please let me know if you'd like me to split this into smaller commits or adjust the structure further.

@vercel
Copy link

vercel bot commented Jan 21, 2026

@yashsarkar164 is attempting to deploy a commit to the Renu's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

🎉 Thanks for the PR, @yashsarkar164!

We really appreciate you taking the time to contribute to OpenPlayground! 💙


⭐ Love this project?

Please give us a star! It helps the project grow and reach more developers! 🌟

🔗 https://github.com/YadavAkhileshh/OpenPlayground


✅ PR Checklist

Before we review, please ensure:

  • Your code follows the project's coding standards
  • All file changes are accurate and intentional
  • You've tested your changes locally
  • Any review comments have been addressed

🚀 Looking for More Ideas & Resources?

Check out Linkshala — your one-stop destination for:

  • 🧩 New project ideas
  • 📚 Learning resources
  • 🎮 Games & creative inspiration
  • 💡 Developer tools & utilities

🔗 https://linkkshala.vercel.app


We'll review your PR as soon as possible. Keep up the great work! ✨

@Renu-code123
Copy link
Owner

Renu-code123 commented Jan 21, 2026

@yashsarkar164, resolve conflict and add a demo video or before-and-after image of the changes.

@Renu-code123
Copy link
Owner

@yashsarkar164 resolve conflict

@vercel
Copy link

vercel bot commented Jan 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
expenseflow Ready Ready Preview, Comment Jan 24, 2026 5:31pm

@Renu-code123
Copy link
Owner

@yashsarkar164 mention issue no.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants